Skip to content

Conversation

@IdrissRio
Copy link
Contributor

See internal PR

@github-actions github-actions bot added the Java label Oct 7, 2025
@IdrissRio IdrissRio added the depends on internal PR This PR should only be merged in sync with an internal Semmle PR label Oct 7, 2025
@IdrissRio IdrissRio marked this pull request as ready for review October 8, 2025 07:50
@IdrissRio IdrissRio requested a review from a team as a code owner October 8, 2025 07:50
Copilot AI review requested due to automatic review settings October 8, 2025 07:50
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds an integration test to verify buildless recovery functionality for Java lambda expressions. The test ensures that CodeQL can properly extract and analyze lambda expressions even when Java compilation fails due to missing imports.

  • Creates a new integration test specifically for lambda expression buildless recovery
  • Tests unused variable detection in various lambda expression scenarios
  • Verifies the extraction errors are handled appropriately during buildless mode

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
unused_variable.ql Query to detect unused local variables in lambda contexts
unused_variable.expected Expected results showing 4 unused variables detected
test.py Test configuration using buildless mode ("none")
Test.java Java source with intentionally missing imports and lambda expressions
ExtractionErrors.qlref Reference to extraction errors diagnostic query
ExtractionErrors.expected Expected extraction errors from buildless compilation

@IdrissRio IdrissRio merged commit 28fe20e into main Oct 8, 2025
10 of 15 checks passed
@IdrissRio IdrissRio deleted the idrissrio/java-lambda branch October 8, 2025 07:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

depends on internal PR This PR should only be merged in sync with an internal Semmle PR Java

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants